FreddyYe marked 2 inline comments as done.
FreddyYe added inline comments.

================
Comment at: llvm/test/CodeGen/X86/cpus-intel.ll:8
 ; RUN: llc < %s -o /dev/null -mtriple=i686-unknown-unknown -mcpu=pentium 2>&1 
| FileCheck %s --check-prefix=CHECK-NO-ERROR --allow-empty
-; RUN: llc < %s -o /dev/null -mtriple=i686-unknown-unknown -mcpu=pentium-mmx 
2>&1 | FileCheck %s --check-prefix=CHECK-NO-ERROR --allow-empty
+; RUN: llc < %s -o /dev/null -mtriple=i686-unknown-unknown -mcpu=pentium-mmx 
-mcpu=pentium_mmx 2>&1 | FileCheck %s --check-prefix=CHECK-NO-ERROR 
--allow-empty
 ; RUN: llc < %s -o /dev/null -mtriple=i686-unknown-unknown -mcpu=i686 2>&1 | 
FileCheck %s --check-prefix=CHECK-NO-ERROR --allow-empty
----------------
RKSimon wrote:
> I meant like this:
> ```
> ; RUN: llc < %s -o /dev/null -mtriple=i686-unknown-unknown -mcpu=pentium-mmx 
> 2>&1 | FileCheck %s --check-prefix=CHECK-NO-ERROR --allow-empty
> ; RUN: llc < %s -o /dev/null -mtriple=i686-unknown-unknown -mcpu=pentium_mmx 
> 2>&1 | FileCheck %s --check-prefix=CHECK-NO-ERROR --allow-empty
> ```
> The idea is to keep RUN lines that test the equivalent cpus together, so its 
> easier for any future edits to handle them together
Woops, I really misunderstood. Updated.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151696/new/

https://reviews.llvm.org/D151696

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to