[PATCH] D44559: [Sema] Wrong width of result of mul operation

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 16 11:36:43 PDT 2018


lebedev.ri added inline comments.


================
Comment at: lib/Sema/SemaChecking.cpp:8541
+      switch (C.getTargetInfo().getTriple().getArch()) {
+      case llvm::Triple::x86:
+      case llvm::Triple::x86_64: {
----------------
I agree with @RKSimon, i don't see why this would be arch-specific.
If this is correct, maybe add explanation as a code comment?


Repository:
  rC Clang

https://reviews.llvm.org/D44559





More information about the cfe-commits mailing list