Cts enable dummy load by deafult
This PR makes the CTS option to insert dummy load buffers at the end of CTS to be on by default.
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
I see a lot of failure that need addressing. A secure CI will be required once those are cleaned up
clang-tidy review says "All clean, LGTM! :+1:"
There are a bunch of conflicts to resolve
clang-tidy review says "All clean, LGTM! :+1:"
All the conflicts were resolved and now there is just one regression design that gets an error during DRT.
Are you referring to aes_sky130hd (tcl) *ERROR* ?
Are you referring to
aes_sky130hd (tcl) *ERROR*?
Yes, I have been talking with Osama on how to solve this.
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
Has this been through secure CI? I don't see a lot of benefit in the OR tests. Is there a good example of the benefit somewhere?
Yes it has been throw a CI, there are a lot of benefits. Cho has a table with the results. However there are 2 designs where the CI is failing and need to be rebased.
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
@precisionmoon Since we only use it at the end and just once I decided to make it an argument, but it can be added to the TritonCTS class if you believe to the best option.
If the usage is very local then I would leave it
clang-tidy review says "All clean, LGTM! :+1:"