llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Timm Baeder (tbaederr)

<details>
<summary>Changes</summary>

There is nothing to fix here. This behavior is on purpose. Remove the "FIXME".

---
Full diff: https://github.com/llvm/llvm-project/pull/196953.diff


2 Files Affected:

- (modified) clang/test/AST/ByteCode/cxx14.cpp (+1-1) 
- (modified) clang/test/SemaCXX/constant-expression-cxx14.cpp (+1-1) 


``````````diff
diff --git a/clang/test/AST/ByteCode/cxx14.cpp 
b/clang/test/AST/ByteCode/cxx14.cpp
index 170bd09504993..97d2c0ac4f711 100644
--- a/clang/test/AST/ByteCode/cxx14.cpp
+++ b/clang/test/AST/ByteCode/cxx14.cpp
@@ -43,7 +43,7 @@ namespace InitListModify {
   };
   constexpr Aggregate aggr1;
   static_assert(aggr1.x == 1 && aggr1.y == 1, "");
-  // FIXME: This is not specified by the standard, but sanity requires it.
+  // This is not specified by the standard, but sanity requires it.
   constexpr Aggregate aggr2 = {};
   static_assert(aggr2.x == 1 && aggr2.y == 1, "");
 }
diff --git a/clang/test/SemaCXX/constant-expression-cxx14.cpp 
b/clang/test/SemaCXX/constant-expression-cxx14.cpp
index fb7fd5b528b05..1bead18080271 100644
--- a/clang/test/SemaCXX/constant-expression-cxx14.cpp
+++ b/clang/test/SemaCXX/constant-expression-cxx14.cpp
@@ -1237,7 +1237,7 @@ namespace ObjectsUnderConstruction {
   };
   constexpr Aggregate aggr1;
   static_assert(aggr1.x == 1 && aggr1.y == 1, "");
-  // FIXME: This is not specified by the standard, but sanity requires it.
+  // This is not specified by the standard, but sanity requires it.
   constexpr Aggregate aggr2 = {};
   static_assert(aggr2.x == 1 && aggr2.y == 1, "");
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/196953
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to