--- a/js/src/builtin/intl/DateTimeFormat.js
+++ b/js/src/builtin/intl/DateTimeFormat.js
@@ -666,33 +666,35 @@ function ToDateTimeOptions(options, requ
     else
         options = ToObject(options);
     options = std_Object_create(options);
 
     // Step 3.
     var needDefaults = true;
 
     // Step 4.
-    // TODO: spec issue - The spec requires to retrieve all options, so using
-    // the ||-operator with its lazy evaluation semantics is incorrect.
-    if ((required === "date" || required === "any") &&
-        (options.weekday !== undefined || options.year !== undefined ||
-         options.month !== undefined || options.day !== undefined))
-    {
-        needDefaults = false;
+    if (required === "date" || required === "any") {
+        if (options.weekday !== undefined)
+            needDefaults = false;
+        if (options.year !== undefined)
+            needDefaults = false;
+        if (options.month !== undefined)
+            needDefaults = false;
+        if (options.day !== undefined)
+            needDefaults = false;
     }
 
     // Step 5.
-    // TODO: spec issue - The spec requires to retrieve all options, so using
-    // the ||-operator with its lazy evaluation semantics is incorrect.
-    if ((required === "time" || required === "any") &&
-        (options.hour !== undefined || options.minute !== undefined ||
-         options.second !== undefined))
-    {
-        needDefaults = false;
+    if (required === "time" || required === "any") {
+        if (options.hour !== undefined)
+            needDefaults = false;
+        if (options.minute !== undefined)
+            needDefaults = false;
+        if (options.second !== undefined)
+            needDefaults = false;
     }
 
     // Step 6.
     if (needDefaults && (defaults === "date" || defaults === "all")) {
         // The specification says to call [[DefineOwnProperty]] with false for
         // the Throw parameter, while Object.defineProperty uses true. For the
         // calls here, the difference doesn't matter because we're adding
         // properties to a new object.

new file mode 100644
--- /dev/null
+++ b/js/src/tests/non262/Intl/DateTimeFormat/options-property-accesses.js
@@ -0,0 +1,71 @@
+// |reftest| skip-if(!this.hasOwnProperty("Intl"))
+
+var log;
+var proxy = new Proxy({
+    year: "numeric",
+    hour: "numeric",
+}, new Proxy({
+    get(t, pk, r) {
+        log.push(pk);
+        return Reflect.get(t, pk, r);
+    }
+}, {
+    get(t, pk, r) {
+        assertEq(pk, "get");
+        return Reflect.get(t, pk, r);
+    }
+}));
+
+var constructorAccesses = [
+    // ToDateTimeOptions(options, "any", "date").
+    "weekday", "year", "month", "day",
+    "hour", "minute", "second",
+
+    // InitializeDateTimeFormat
+    "localeMatcher", "hour12", "hourCycle", "timeZone",
+
+    // Table 5: Components of date and time formats
+    "weekday", "era", "year", "month", "day", "hour", "minute", "second", "timeZoneName",
+
+    // InitializeDateTimeFormat
+    "formatMatcher",
+];
+
+log = [];
+new Intl.DateTimeFormat(undefined, proxy);
+
+assertEqArray(log, constructorAccesses);
+
+log = [];
+new Date().toLocaleString(undefined, proxy);
+
+assertEqArray(log, [
+    // ToDateTimeOptions(options, "any", "all").
+    "weekday", "year", "month", "day",
+    "hour", "minute", "second",
+
+    ...constructorAccesses
+]);
+
+log = [];
+new Date().toLocaleDateString(undefined, proxy);
+
+assertEqArray(log, [
+    // ToDateTimeOptions(options, "date", "date").
+    "weekday", "year", "month", "day",
+
+    ...constructorAccesses
+]);
+
+log = [];
+new Date().toLocaleTimeString(undefined, proxy);
+
+assertEqArray(log, [
+    // ToDateTimeOptions(options, "time", "time").
+    "hour", "minute", "second",
+
+    ...constructorAccesses
+]);
+
+if (typeof reportCompare === "function")
+    reportCompare(0, 0);